Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

20230211-wolfcrypttest-fixes #6081

Merged
merged 1 commit into from
Feb 13, 2023

Conversation

douzzer
Copy link
Contributor

@douzzer douzzer commented Feb 11, 2023

wolfcrypt/test/test.c: around ecc_ctx_kdf_salt_test(): fix fips gating, fix length handling for message;

in crypto_ecc_verify(), crypto_ecc_sign(), ecc_test_nonblock_dhe(), and ecc_test_nonblock_ecdsa(), add codepoint-specific retvals.

tested with wolfssl-multi-test.sh ... check-file-modes check-source-text check-shell-scripts all-gcc-c99 fips-140-3-all linuxkm-all-fips-140-3 linuxkm-all-fips-140-3-dyn-hash linuxkm-defaults-all-fips-140-3 fips-140-3-RC12 clang-tidy-asn-template-sp-all-small-stack all-WOLFSSL_CALLBACKS-clang-tidy clang-tidy-all-sp-all clang-tidy-all-async-quic clang-tidy-fips-140-3-all clang-tidy-fips-140-3-dev-all clang-tidy-fips-140-3-dev-all-crypto-no-sha-1 clang-tidy-all-crypto-no-sha-1 sp-all-asm-smallstack-valgrind-testsuite

@douzzer
Copy link
Contributor Author

douzzer commented Feb 12, 2023

note the Fips Check failure is a bug in master related to #5952

@douzzer
Copy link
Contributor Author

douzzer commented Feb 13, 2023

closed in favor of #6083

@douzzer douzzer closed this Feb 13, 2023
@douzzer douzzer reopened this Feb 13, 2023
…g, fix length handling for "message", fix memory leaks;

in crypto_ecc_verify(), crypto_ecc_sign(), ecc_test_nonblock_dhe(), and ecc_test_nonblock_ecdsa(), add codepoint-specific retvals.
@dgarske dgarske merged commit 6877c98 into wolfSSL:master Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants